-
-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Deployment: Dockerfile and Smithery config #2
Conversation
Thank you for your PR! I attempted to build the Docker image, but I encountered the following error:
Could you confirm if I'm missing anything in the setup or build step? |
Hmm there might be a bug in the generated Dockerfile. Could you try removing this line?
If that still doesn't work, perhaps we can try a single-stage build (less efficient, but should work)? |
Removing that line resolved the issue for me. Thank you! I will proceed to update and merge this PR. |
@calclavia I tried running the following command:
However, I encountered this error:
Am I missing something? |
You have to deploy on Smithery first before the CLI command will work. Otherwise, we don't have your server served up for clients yet. |
I see. So the |
Not for now. We're developing a way to pull servers and run it locally so it'll support both local and remote servers in the next few months. It's WIP! |
This pull request introduces the following updates:
Please review these updates to verify their accuracy for your server and feel free to customize it to your needs. Let me know if you have any questions. 🙂